Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PWX-32825: Adjusting version triggers for px-3.0.1 #1207

Merged
merged 1 commit into from
Aug 17, 2023

Conversation

zoxpx
Copy link
Collaborator

@zoxpx zoxpx commented Aug 15, 2023

What this PR does / why we need it:

ISSUE:

  • our QA used image-override 3.0.1-ubuntu1604 -- and this BROKE the version-checks
NewVersion("3.0.1-ubuntu1604").GreaterThanOrEqual("3.0.1")  // this evaluates as FALSE!!
  • note, the "version with extension" (such as 3.0.1-ubuntu1604), will evaluate similar like 3.0.1.1 (a version higher than 3.0.1)

FIX:

  • as a fix, we're changing the version.GreaterThanOrEqual("3.0.1") into version.GreaterThan("3.0")

Which issue(s) this PR fixes (optional)
Closes # PWX-32825

Special notes for your reviewer:

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (332a15b) 75.77% compared to head (a08d013) 75.77%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1207   +/-   ##
=======================================
  Coverage   75.77%   75.77%           
=======================================
  Files          65       65           
  Lines       18410    18410           
=======================================
  Hits        13951    13951           
  Misses       3464     3464           
  Partials      995      995           
Files Changed Coverage Δ
drivers/storage/portworx/deployment.go 95.91% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@zoxpx
Copy link
Collaborator Author

zoxpx commented Aug 15, 2023

Travis failing due to unrelated error --

=== RUN   TestUpdateStorageNodeKVDB
    util.go:4849: This test requires ROOT user  (writeable /etc/hosts): permission denied
time="2023-08-15T06:09:57Z" level=fatal msg="ERROR: Unable to start test gRPC server: grpc: the server has been stopped\n"
FAIL	github.com/libopenstorage/operator/drivers/storage/portworx	51.793s
FAIL
make: *** [Makefile:147: test] Error 1

Will rebase + retest

@zoxpx zoxpx force-pushed the PWX-32825_adjusting_version_triggers branch from a2ee82b to cb71ce2 Compare August 15, 2023 16:44
@harsh-px
Copy link
Contributor

Change looks good. But we need to resolve conflicts before merge.

Signed-off-by: Zoran Rajic <zox@portworx.com>
@zoxpx zoxpx force-pushed the PWX-32825_adjusting_version_triggers branch from cb71ce2 to a08d013 Compare August 17, 2023 17:25
@zoxpx
Copy link
Collaborator Author

zoxpx commented Aug 17, 2023

Thanks for the review Harsh -- merging this PR, will cherry-pick into px-rel-23.7.0

@zoxpx zoxpx merged commit 5295601 into master Aug 17, 2023
8 checks passed
@zoxpx zoxpx deleted the PWX-32825_adjusting_version_triggers branch August 17, 2023 18:43
zoxpx added a commit that referenced this pull request Aug 17, 2023
Signed-off-by: Zoran Rajic <zox@portworx.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants